[PATCH] D78517: [PC Relative][Expolitation] Remove "unskipableSimplifyCode()" in PPCMIPeephole.cpp

[Stefan Pintilie via Phabricator via llvm-commits]

stefanp accepted this revision.
stefanp added a comment.
This revision is now accepted and ready to land.

Minor wording as Anil pointed out.
LGTM.



================
Comment at: llvm/lib/Target/PowerPC/PPCMIPeephole.cpp:128
     initialize(MF);
-    // FIXME: This introduces another complete traversal of the instructions
-    // in the function in the common case (function is not skipped). Although
-    // this is less than ideal for compile time, this code will go away once
-    // our PC-Rel implementation is complete.
-    unskipableSimplifyCode();
+    // At this point, R2 can be only used as the TOC pointer, which should not
+    // be found when using PC Relative.
----------------
anil9 wrote:
> nit : For both the comment and the message in the assert. Ending the sentences with just "PC Relative" seems incorrect, may be use PC Relative instructions/functionality/operations. I see many asserts ending with or without full stops and some with exclamation sign. Not sure which one is correct.
You could use `PC Relative addressing` because that's what it really is. It is a way to generate the address of something without using the TOC pointer in R2.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78517