[PATCH] D73301: [PatchableFunction] Allow empty entry MachineBasicBlock

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 24 09:55:26 PST 2020


MaskRay added a subscriber: hans.
MaskRay added a comment.

@hans OK to cherry pick this commit? We would also need d232c215669cb57f5eb4ead40a4a336220dbc429 <https://reviews.llvm.org/rGd232c215669cb57f5eb4ead40a4a336220dbc429> ([AsmPrinter] Don't emit __patchable_function_entries entry if "patchable-function-entry"="0"; a GCC compatibility issue I did not notice when I implemented -fpatchable-function-entry=N,0).

If users want to see a complete -fpatchable-function-entry=N,M, instead of just -fpatchable-function-entry=N,0 , in clang 10.0.0, we will need D73070 <https://reviews.llvm.org/D73070>, D73071 <https://reviews.llvm.org/D73071>, and D73072 <https://reviews.llvm.org/D73072> as well (N,M where M>0 is currently not used, though someone told me that Linux arch/arm64 may want to do that in the future. If that is the case, supporting a complete N,M, not just N,0 can simplify Linux kernel side configure test.)


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D73301/new/

https://reviews.llvm.org/D73301





More information about the llvm-commits mailing list