This is an archive of the discontinued LLVM Phabricator instance.

[WebAssembly] Remove unneeded target operand flags
ClosedPublic

Authored by sbc100 on Mar 29 2019, 2:40 PM.

Details

Summary

This change is in preparation for the addition of new target
operand flags for new relocation types:

https://reviews.llvm.org/D59907

Having the symbol type as part of the flag set makes it harder
to use and AFAICT these are serving no purpose.

Event Timeline

sbc100 created this revision.Mar 29 2019, 2:40 PM
Herald added a project: Restricted Project. · View Herald TranscriptMar 29 2019, 2:40 PM
sbc100 edited the summary of this revision. (Show Details)Mar 29 2019, 2:49 PM
sbc100 added a reviewer: dschuff.
sbc100 updated this revision to Diff 192922.Mar 29 2019, 2:50 PM
sbc100 edited the summary of this revision. (Show Details)
  • remove fixed value
dschuff accepted this revision.Apr 2 2019, 4:50 PM

I think these are leftovers from when we were using ELF.

This revision is now accepted and ready to land.Apr 2 2019, 4:50 PM
This revision was automatically updated to reflect the committed changes.