nikic added a comment.

@nathanchance Thanks for the report. I've applied the obvious fix in 
https://github.com/llvm/llvm-project/commit/20962c1240691d25b21ce425313c81eed0b1b358.
 However, I think that this (and similar limitations relating to critical edge 
splitting) might actually be unnecessary since 
https://github.com/llvm/llvm-project/commit/7a7bba289521e6d4da199565cf72dc9eaed3d671.
 callbr has stricter semantics than indirectbr, and apparently updating 
successors of a callbr //is// supposed to be possible (because the 
blockaddresses used by it must be directly used in the call arguments and thus 
can be updated, while in the indirectbr case they may be passed indirectly).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D124159

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to